Skip to content

Pull request for unit and integration tests#2

Merged
Amynka merged 18 commits into
masterfrom
matthias
Jun 9, 2026
Merged

Pull request for unit and integration tests#2
Amynka merged 18 commits into
masterfrom
matthias

Conversation

@Matthjass13

Copy link
Copy Markdown
Collaborator

Add 40 unit tests
Add 44 integration tests
Add pictures of the 3 scenarios used for documentation
A few corrections to the framework

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread README.md Outdated
/*
This break statement is required in order to stop the program at the right time.
Without it, the program would keep exploring the current queue until it is empty,
which is not what we want I guess ??

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this line

cpt <= options.getCuttOfPlanning()){
if(options.getTypeOfCuttOfPlanning().equals("INT")
//&&idCpt <= options.getCuttOfPlanning())
//cpt <= options.getCuttOfPlanning())

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this and reformat so its readable.. Both comments not needed

@Amynka Amynka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@Amynka Amynka merged commit 654b8fb into master Jun 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants